Skip to content

Comments

fix: resolve deadlock when both sides send resendrequest simultaneously#314

Merged
davidsteiner merged 6 commits intomainfrom
312-deadlock-when-both-sides-send-resendrequest-simultaneously
Feb 24, 2026
Merged

fix: resolve deadlock when both sides send resendrequest simultaneously#314
davidsteiner merged 6 commits intomainfrom
312-deadlock-when-both-sides-send-resendrequest-simultaneously

Conversation

@davidsteiner
Copy link
Member

No description provided.

@davidsteiner davidsteiner linked an issue Feb 23, 2026 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Feb 23, 2026

Codecov Report

❌ Patch coverage is 89.84772% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.90%. Comparing base (5491727) to head (1ed7b3d).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
crates/hotfix/src/session.rs 80.88% 13 Missing ⚠️
crates/hotfix/src/message.rs 76.00% 6 Missing ⚠️
crates/hotfix/src/session/state.rs 90.90% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #314      +/-   ##
==========================================
+ Coverage   90.73%   90.90%   +0.16%     
==========================================
  Files          86       85       -1     
  Lines        8732     8783      +51     
==========================================
+ Hits         7923     7984      +61     
+ Misses        809      799      -10     
Flag Coverage Δ
core 90.77% <89.84%> (+0.17%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@davidsteiner davidsteiner merged commit 5eef99a into main Feb 24, 2026
6 checks passed
@davidsteiner davidsteiner deleted the 312-deadlock-when-both-sides-send-resendrequest-simultaneously branch February 24, 2026 15:49
@github-actions github-actions bot mentioned this pull request Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deadlock when both sides send ResendRequest simultaneously

2 participants